[TOO-314] Tool documentation auto generation tool and replacing hardcode mdx for component based one #705
+545,932
−718,865
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ode mdx for component based one
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
app/_components/toolkit-docs/components/AvailableToolsTable.tsx
Outdated
Show resolved
Hide resolved
app/_components/toolkit-docs/components/documentation-chunk-renderer.tsx
Show resolved
Hide resolved
app/_components/toolkit-docs/components/AvailableToolsTable.tsx
Outdated
Show resolved
Hide resolved
- Update layout.tsx with locale detection from pathname - Update globals.css with nextra-steps styling - Update robots.txt with AI Agent Resources comment - Keep gmail integration page from main
Resolve all merge conflicts: - Keep our scope-picker.tsx with PostHog tracking - Keep our not-found.tsx with Suspense boundary fix - Keep our package.json with toolkit scripts and mdx deps - Keep next-sitemap.config.js (deleted in main) - Merge globals.css (keyframes + nextra-steps styling) - Merge gitignore (our additions)
app/_components/toolkit-docs/components/documentation-chunk-renderer.tsx
Show resolved
Hide resolved
…y curser (yes, i wil blame him)
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| list.push(toolkit); | ||
| } else { | ||
| grouped.set(category, [toolkit]); | ||
| } |
There was a problem hiding this comment.
Unreachable else branch in groupByCategory function
Low Severity
The else branch in the groupByCategory function is unreachable dead code. The function first checks if (!grouped.has(category)) and sets an empty array if the category doesn't exist. It then immediately retrieves the list with grouped.get(category). Since the category is guaranteed to exist after the first conditional, list will always be defined, making the else branch impossible to reach.
Additional Locations (1)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Note
Medium Risk
Medium risk because it adds new automation (GitHub Actions + a filesystem-writing sync script) and introduces new client-side MDX evaluation/caching and UI logic that could affect docs rendering/performance.
Overview
Adds a new automation path to regenerate toolkit JSON after Porter deploys and open a PR, including a new
.github/scripts/sync-toolkit-sidebar.ts(with tests/docs) that derives integrations_meta.tsxnavigation fromdata/toolkits/and design-system categories.Introduces a new component-based toolkit docs surface (
app/_components/toolkit-docs/*) including searchable/paginated tools tables, per-tool sections, dynamic code example generation, and MDX-driven documentation chunk injection with caching; updatesScopePickerto support controlled selection, pagination, secrets/scopes aggregation, and copy actions.Smaller updates: dashboard link URL/prefix normalization, use toolkit icon URLs instead of component lookup, and repo/editor ignore settings to reduce watcher load.
Written by Cursor Bugbot for commit 04aa22d. This will update automatically on new commits. Configure here.